Fixes Foundry Hosted Agents HTTP endpoints by tommasodotNET · Pull Request #16130 · microsoft/aspire

tommasodotNET · 2026-04-13T21:06:23Z

Description

Closes #16115. Overwrites existing http/https endpoints on resource to allow the creation of dedicated hosted agent endpoint. Does not hard code http port (letting aspire handle port assignment). PublishAsHostedAgent accepts generic resource with endpoint (e.g. ProjectResource) allowing hosted agents in dotnet (see also #15968).
Allows for custom targetPorts to allow locally run multiple hosted agents

Adds a playground sample for hosted agents in python

Fixes # (issue)

Checklist

Is this feature complete?
- Yes. Ready to ship.
- No. Follow-up changes expected.
Are you including unit tests for the changes and scenario tests if relevant?
- Yes
- No
Did you add public API?
- Yes
  - If yes, did you have an API Review for it?
    - Yes
    - No
  - Did you add <remarks /> and <code /> elements on your triple slash comments?
    - Yes
    - No
- No
Does the change make any security assumptions or guarantees?
- Yes
  - If yes, have you done a threat model and had a security review?
    - Yes
    - No
- No
Does the change require an update in our Aspire docs?
- Yes
  - Link to aspire.dev issue:
    - New issue
- No

github-actions · 2026-04-13T21:13:35Z

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 16130

Or

Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 16130"

…ts and environment interfaces

Copilot

Pull request overview

Updates the Foundry hosted-agent run-mode behavior to avoid endpoint conflicts when PublishAsHostedAgent is applied to resources that already define HTTP endpoints, and adds a new playground sample for hosted agents.

Changes:

In run mode, removes existing HTTP/HTTPS endpoint annotations before adding the hosted-agent HTTP endpoint.
Refactors publish-mode containerization to explicitly handle ProjectResource vs ExecutableResource.
Adds a playground/FoundryHostedAgents sample (AppHost + Python agent app scaffolding).

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/Aspire.Hosting.Foundry.Tests/HostedAgentExtensionTests.cs	Adds tests asserting hosted-agent run-mode endpoint overwrite behavior and non-hardcoded port behavior.
src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExtension.cs	Implements endpoint removal in run mode and adjusts publish-mode containerization logic.
playground/FoundryHostedAgents/aspire.config.json	Adds Aspire config pointing to the new playground AppHost.
playground/FoundryHostedAgents/app/pyproject.toml	Adds Python agent sample dependencies/config.
playground/FoundryHostedAgents/app/main.py	Adds Python agent sample implementation.
playground/FoundryHostedAgents/app/.dockerignore	Adds Docker ignore rules for the Python sample.
playground/FoundryHostedAgents/FoundryHostedAgents.AppHost/appsettings.json	Adds logging configuration for the new AppHost sample.
playground/FoundryHostedAgents/FoundryHostedAgents.AppHost/appsettings.Development.json	Adds development Azure settings for the sample (currently includes specific identifiers).
playground/FoundryHostedAgents/FoundryHostedAgents.AppHost/Properties/launchSettings.json	Adds launch profiles for the new AppHost sample.
playground/FoundryHostedAgents/FoundryHostedAgents.AppHost/FoundryHostedAgents.AppHost.csproj	Adds a new AppHost project file for the playground sample.
playground/FoundryHostedAgents/FoundryHostedAgents.AppHost/AppHost.cs	Adds the new playground AppHost wiring for Foundry + Python hosted agent.

…settings.Development.json Co-authored-by: Copilot <[email protected]>

…m:tommasodotNET/aspire into fix/hosted-agent-overwrite-http-endpoints

…sion.cs Co-authored-by: Copilot <[email protected]>

…m:tommasodotNET/aspire into fix/hosted-agent-overwrite-http-endpoints

tommasodotNET · 2026-04-14T07:27:14Z

UPDATE: instead of removing the existing endpoints, we can overwrite them. looks like a better solution. The fix still enables use with dotnet hosted agents (sample added).
We also allow custom targetPorts to handle multiple local hosted agents

…target ports

…st stable releases

…erence

eerhardt · 2026-04-14T14:10:09Z

+
+var tenantId = builder.AddParameterFromConfiguration("tenant", "Azure:TenantId");
+
+var foundry = builder.AddFoundry("aif-myfoundry");


We should consider making a deployment end-to-end test out of this.

See https://github.com/microsoft/aspire/tree/main/.github/skills/deployment-e2e-testing and https://github.com/microsoft/aspire/tree/main/tests/Aspire.Deployment.EndToEnd.Tests

i've tried adding one. not sure if it's correct, though

…ntBuilderExtension

…package versions to 1.1.0 and 10.4.0 respectively

davidfowl · 2026-04-14T16:52:25Z

+        Assert.True(app.Resource.TryGetEndpoints(out var endpoints));
+        var httpEndpoints = endpoints.Where(e => e.Name == "http").ToList();
+        Assert.Single(httpEndpoints);
+        Assert.Equal(8088, httpEndpoints[0].TargetPort);


? shouldn't this be 5000

PublishAsHostedAgent overwrites the http endpoints on the resource. this is needed because you could have a generic PythonApp hosting an agent with microsoft agent framework.
but the the user can explicitly set his own target port, adding .WithHttpEndpoint() after invoking PublishAsHostedAgent().

so this test is verifying that whichever previously defined http endpoint gets overwritten to standard foundry hosted agent default value 8088.

it's not ideal, tbh. the thing is hosting the agent for foundry via microsoft agent framework usually expose the agent at 8088 as default, unless otherwise specified in the DEFAULT_AD_PORT internal env var. so to map different targetPorts you can do what i did in the playground in this pr (look at the dotnet app).

i'm working on a fix for this scenario btw. commit incoming. if an "http" endpoint already exists, we use the target port defined in that one, otherwise we default to 8088

…d agent configuration

Remove existing HTTP/HTTPS endpoints before adding new ones in run mode

a1506c4

tommasodotNET requested a review from eerhardt April 13, 2026 21:07

Tommaso Stocchi added 3 commits April 13, 2026 23:52

include sample for foundry hosted agents

4a66ae7

Refactor PublishAsHostedAgent method to accept resources with endpoin…

abbb568

…ts and environment interfaces

updates foundry names

b4ea257

tommasodotNET marked this pull request as ready for review April 13, 2026 21:54

tommasodotNET requested a review from sebastienros as a code owner April 13, 2026 21:54

Copilot AI review requested due to automatic review settings April 13, 2026 21:54

Copilot started reviewing on behalf of tommasodotNET April 13, 2026 21:58 View session

Copilot AI reviewed Apr 13, 2026

View reviewed changes

sebastienros reviewed Apr 13, 2026

View reviewed changes

Comment thread tests/Aspire.Hosting.Foundry.Tests/HostedAgentExtensionTests.cs Outdated

tommasodotNET and others added 7 commits April 14, 2026 08:45

Update playground/FoundryHostedAgents/FoundryHostedAgents.AppHost/app…

2f9be79

…settings.Development.json Co-authored-by: Copilot <[email protected]>

Fix: Update HTTP endpoint proxying behavior in run mode test

038a43d

Merge branch 'fix/hosted-agent-overwrite-http-endpoints' of github.co…

801c8fa

…m:tommasodotNET/aspire into fix/hosted-agent-overwrite-http-endpoints

Update src/Aspire.Hosting.Foundry/HostedAgent/HostedAgentBuilderExten…

2793d22

…sion.cs Co-authored-by: Copilot <[email protected]>

adds dotnet example

c52d654

Merge branch 'fix/hosted-agent-overwrite-http-endpoints' of github.co…

541b4e1

…m:tommasodotNET/aspire into fix/hosted-agent-overwrite-http-endpoints

reverts foundry names in sample

d0a5b80

Tommaso Stocchi added 2 commits April 14, 2026 10:09

allow multiple hosted agents to run locally by configuring different …

5b02382

…target ports

update Microsoft.Agents.AI.OpenAI and OpenAI package versions to late…

dd27397

…st stable releases

tommasodotNET changed the title ~~Remove existing HTTP/HTTPS endpoints before adding new ones in run mode~~ Fixes Foundry Hosted Agents HTTP endpoints Apr 14, 2026

tommasodotNET requested a review from sebastienros April 14, 2026 08:52

Tommaso Stocchi added 2 commits April 14, 2026 11:06

add Azure.AI.AgentServer package source and remove unused project ref…

4f41344

…erence

set DefaultTargetFramework on samples

cc944ce

tommasodotNET mentioned this pull request Apr 14, 2026

Foundry Hosted Agent responses are rendered weirdly #16129

Open

1 task

eerhardt reviewed Apr 14, 2026

View reviewed changes

sebastienros reviewed Apr 14, 2026

View reviewed changes

Comment thread Directory.Packages.props Outdated

fixes on the playground

a92606b

Tommaso Stocchi added 4 commits April 14, 2026 17:19

refactor: streamline container resource deployment logic in HostedAge…

e187d4b

…ntBuilderExtension

-fix apphost var ref

85b4a41

chore: update Microsoft.Agents.AI.OpenAI and Microsoft.Extensions.AI …

5aa7773

…package versions to 1.1.0 and 10.4.0 respectively

adds end2end test

f320db2

davidfowl reviewed Apr 14, 2026

View reviewed changes

Comment thread tests/Aspire.Hosting.Foundry.Tests/HostedAgentExtensionTests.cs Outdated

davidfowl reviewed Apr 14, 2026

View reviewed changes

github-actions bot mentioned this pull request Apr 14, 2026

[pull requests] Review required #13834

Open

Tommaso Stocchi added 3 commits April 14, 2026 22:25

fix: preserve existing HTTP endpoint target port in run mode

094aea5

fix: reorder method calls for clarity in hosted agent configuration

8af2bb5

fix: reorder method calls to ensure proper reference waiting in hoste…

4a7bc24

…d agent configuration


		var tenantId = builder.AddParameterFromConfiguration("tenant", "Azure:TenantId");

		var foundry = builder.AddFoundry("aif-myfoundry");

Conversation

tommasodotNET commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Uh oh!

github-actions bot commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tommasodotNET commented Apr 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

eerhardt Apr 14, 2026

Choose a reason for hiding this comment

Uh oh!

tommasodotNET Apr 14, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

davidfowl Apr 14, 2026

Choose a reason for hiding this comment

Uh oh!

tommasodotNET Apr 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tommasodotNET Apr 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

tommasodotNET commented Apr 13, 2026 •

edited

Loading

github-actions bot commented Apr 13, 2026 •

edited

Loading

tommasodotNET commented Apr 14, 2026 •

edited

Loading

tommasodotNET Apr 14, 2026 •

edited

Loading

tommasodotNET Apr 14, 2026 •

edited

Loading